-
Notifications
You must be signed in to change notification settings - Fork 90
Add case for 'Pimple\Psr11\Container' in HP #270
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this is a bugfix, we'd need a test case for it
Hello @Ocramius @xtreamwayz, (*edited) ...or just skip other containers when using zend-view? The |
Ok, skipping the zend-view / non zend-sm combinations solved it. I was wondering if we could just use php imports and |
This repository has been closed and moved to mezzio/mezzio-skeleton; a new issue has been opened at mezzio/mezzio-skeleton#4. |
This repository has been moved to mezzio/mezzio-skeleton. If you feel that this patch is still relevant, please re-open against that repository, and reference this issue. To re-open, we suggest the following workflow:
|
Add case for new Pimple containerName in HomePageHandler